[Vlans]: Extract VLAN_MEMBER_TABLE from VLAN_TABLE#229
Merged
pavel-shirshov merged 3 commits intosonic-net:masterfrom Jun 1, 2017
Merged
[Vlans]: Extract VLAN_MEMBER_TABLE from VLAN_TABLE#229pavel-shirshov merged 3 commits intosonic-net:masterfrom
pavel-shirshov merged 3 commits intosonic-net:masterfrom
Conversation
lguohan
reviewed
Jun 1, 2017
doc/swss-schema.md
Outdated
| mtu = 1*4DIGIT ; MTU for the IP interface of the VLAN | ||
|
|
||
| key = VLAN_TABLE:vlanid:ifname ; physical port member of VLAN | ||
| key = VLAN_MEMBER_TABLE:vlanid:ifname ; physical port member of VLAN |
Contributor
There was a problem hiding this comment.
can you add a separate table here?
For the VLAN_MEMBER_TABLE, tagging_mode is its attribute.
For the VLAN_TABLE, it does not need to have tagging_mode as an attribute.
Contributor
There was a problem hiding this comment.
In the current orchagent code, it does not read this tagging mode. You need this function, I think you can do that in another separate CR.
lguohan
approved these changes
Jun 1, 2017
Contributor
lguohan
left a comment
lguohan
left a comment
There was a problem hiding this comment.
seperate the VLAN_MEMBER_TABLE in the swss-schema.md
EdenGri
pushed a commit
to EdenGri/sonic-swss
that referenced
this pull request
Feb 28, 2022
…#229) Signed-off-by: Volodymyr Samotiy <[email protected]>
oleksandrivantsiv
pushed a commit
to oleksandrivantsiv/sonic-swss
that referenced
this pull request
Mar 1, 2023
* [syncd][PFC WD]: Add support for SAI PFC storm notification This commit implements native support for SAI_SWITCH_ATTRIBUTE_QUEUE_PFC_DEADLOCK_NOTIFY. Calling lua scripts is moved from orchagent to syncd, which uses native notification by a SW watchdog as well. Signed-off-by: marian-pritsak <[email protected]> * [syncd_pfcwatchdog]: Add plugin subscription Signed-off-by: marian-pritsak <[email protected]> * [syncd]: Remove native notidications Signed-off-by: marian-pritsak <[email protected]> * [syncd.cpp]: Fix typo Signed-off-by: marian-pritsak <[email protected]> * Address comments Signed-off-by: marian-pritsak <[email protected]>
lukasstockner
pushed a commit
to genesiscloud/sonic-swss
that referenced
this pull request
Apr 2, 2023
…SD info (sonic-net#229) The problem: The current SSD health API uses a SSD vendor specific utility to retrieve values for overall SSD health and internal SSD temperature. These utilities are typically made available only in binary form and are not always available for all platform architectures (amd64, arm64, armhf, etc.) Vendor licensing agreements can prevent the inclusion of utility source code in the Sonic build. The solution presented here is to retrieve SSD health and temperature values from the generic SSD info retrieved via the open-source smartmontools utility (smartctl). These values can be overwritten with values retrieved using a vendor specific utility when available. smartmontools is already included in the Sonic source tree and is being built for all platform architectures.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
127.0.0.1:6379> keys VLAN*